wayland: Allow a NULL inhibitors hash table
authorEmmanuele Bassi <ebassi@gnome.org>
Mon, 23 Apr 2018 09:57:07 +0000 (10:57 +0100)
committerMatthias Clasen <mclasen@redhat.com>
Mon, 23 Apr 2018 22:54:43 +0000 (18:54 -0400)
commit3f307c59bd0f4e746f1ce30bfb1abf8036c4eeb2
tree520dd99758ebe37d12cc356f55e15439f8b16caf
parent2a87caf529a76ae8d1a8a7b72801c47e0350b68e
wayland: Allow a NULL inhibitors hash table

The shortcuts inhibitors hash table is created when we create a
GdkWaylandWindow implementation for a GdkWindow, and it's destroyed once
we finalize the instance. The fake "root" window we create for the
Wayland display does not have a backing native window, so the shortcuts
inhibitors hash table is set to NULL; this causes a critical error
message when calling g_hash_table_destroy() on it. The finalization of
the root window happens when we close a display connection.

We should use g_clear_pointer(), instead, as it's NULL safe.

Without this change, the displayclose test fails, as all warnings are
considered fatal.
gdk/wayland/gdksurface-wayland.c